[wip] feat: vue 3 upgrade (VMO-6873) #339
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
| // eslint-disable-next-line @typescript-eslint/no-explicit-any | ||
| export default function registerCustomComponents(extra = {}): Record<string, any> { | ||
| export default function registerCustomComponents(app, extra = {}) { |
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.@typescript-eslint/explicit-function-return-type
Severity: WARN
File: src/common-imports.ts L19
Missing return type on function. (@typescript-eslint/explicit-function-return-type)
| @@ -11,11 +9,14 @@ import * as toolbarComponents from '@/components/interaction-designer/toolbar' | |||
| import * as blocksComponents from '@/components/interaction-designer/blocks' | |||
| import * as blockResourceEditorsComponents from '@/components/interaction-designer/resource-editors' | |||
| import * as resourceEditorComponents from '@/components/resource-editor' | |||
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.import/newline-after-import
Severity: ERROR
File: src/common-imports.ts L11
Expected 1 empty line after import statement not followed by another import. (import/newline-after-import)
| @@ -78,15 +78,15 @@ | |||
| class="col-md-6" | |||
| @click.prevent="decrementPage"> | |||
| <i class="glyphicon glyphicon-chevron-left" /> | |||
| {{ 'flow-builder.previous' | trans }} | |||
| {{ trans('flow-builder.previous') }} | |||
| </a> | |||
|
|
|||
| <a | |||
| href="#" | |||
| :class="{disabled: !hasNext}" | |||
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.vue/object-curly-spacing
Severity: ERROR
File: src/components/common/AudioLibrarySearchField.vue L86
A space is required after '{'. (vue/object-curly-spacing)
| @@ -78,15 +78,15 @@ | |||
| class="col-md-6" | |||
| @click.prevent="decrementPage"> | |||
| <i class="glyphicon glyphicon-chevron-left" /> | |||
| {{ 'flow-builder.previous' | trans }} | |||
| {{ trans('flow-builder.previous') }} | |||
| </a> | |||
|
|
|||
| <a | |||
| href="#" | |||
| :class="{disabled: !hasNext}" | |||
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.vue/object-curly-spacing
Severity: ERROR
File: src/components/common/AudioLibrarySearchField.vue L86
A space is required before '}'. (vue/object-curly-spacing)
| @@ -4,7 +4,7 @@ | |||
| <input | |||
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.vuejs-accessibility/form-control-has-label
Severity: ERROR
File: src/components/common/AudioLibrarySearchField.vue L4
Each form element must have a programmatically associated label element. (vuejs-accessibility/form-control-has-label)
| @@ -27,7 +27,7 @@ | |||
| class="dropdown-item" | |||
| :href="audioFileUrl" | |||
| target="_blank"> | |||
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.vue/no-template-target-blank
Severity: ERROR
File: src/components/common/AudioLibrarySelection.vue L29
Using target="_blank" without rel="noopener noreferrer" is a security risk. (vue/no-template-target-blank)
| import {Getter, namespace} from 'vuex-class' | ||
| import {MethodNodeEvaluatorFactory} from '@floip/expression-evaluator/dist/Evaluator/NodeEvaluator/MethodNodeEvaluator/Factory' | ||
| import Lang from '@/lib/filters/lang' | ||
| import {Lang} from '@/lib/filters/lang' | ||
| import {IBlock, IFlow} from '@floip/flow-runner' |
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.import/order
Severity: ERROR
File: src/components/common/ExpressionInput.vue L39
`@floip/flow-runner` import should occur before import of `@/lib/filters/lang` (import/order)
| import PlainDraggableLib from 'plain-draggable' | ||
| import Lang from '@/lib/filters/lang' | ||
| import {Lang} from '@/lib/filters/lang' | ||
| import {IPositionLeftTop} from '@/lib/types' | ||
| import {namespace} from 'vuex-class' |
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.import/order
Severity: ERROR
File: src/components/common/PlainDraggable.vue L13
`vuex-class` import should occur before import of `@/lib/filters/lang` (import/order)
| import {BAlert} from 'bootstrap-vue' | ||
| import {Prop} from 'vue-property-decorator' | ||
| import {mixins, Options} from 'vue-class-component' | ||
| import {Lang} from '@/lib/filters/lang' |
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.import/order
Severity: ERROR
File: src/components/common/ValidationMessage.vue L19
`@/lib/filters/lang` import should occur after import of `vuex-class` (import/order)
| @@ -137,7 +138,7 @@ export class BuilderCanvas extends Vue { | |||
| return temporaryDummyWidth | |||
| } | |||
|
|
|||
| return ((blockElementRef[0].$refs.draggable as Vue).$el as HTMLElement).offsetWidth | |||
| return ((blockElementRef[0].$refs.draggable as any).$el as HTMLElement).offsetWidth | |||
| } | |||
|
|
|||
| get blockAtTheLowestPosition(): any { | |||
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.@typescript-eslint/no-explicit-any
Severity: WARN
File: src/components/interaction-designer/BuilderCanvas.vue L144
Unexpected any. Specify a different type. (@typescript-eslint/no-explicit-any)
| @@ -137,7 +138,7 @@ export class BuilderCanvas extends Vue { | |||
| return temporaryDummyWidth | |||
| } | |||
|
|
|||
| return ((blockElementRef[0].$refs.draggable as Vue).$el as HTMLElement).offsetWidth | |||
| return ((blockElementRef[0].$refs.draggable as any).$el as HTMLElement).offsetWidth | |||
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.@typescript-eslint/no-unnecessary-type-assertion
Severity: ERROR
File: src/components/interaction-designer/BuilderCanvas.vue L141
This assertion is unnecessary since it does not change the type of the expression. (@typescript-eslint/no-unnecessary-type-assertion)
| @@ -137,7 +138,7 @@ export class BuilderCanvas extends Vue { | |||
| return temporaryDummyWidth | |||
| } | |||
|
|
|||
| return ((blockElementRef[0].$refs.draggable as Vue).$el as HTMLElement).offsetWidth | |||
| return ((blockElementRef[0].$refs.draggable as any).$el as HTMLElement).offsetWidth | |||
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.@typescript-eslint/no-explicit-any
Severity: WARN
File: src/components/interaction-designer/BuilderCanvas.vue L141
Unexpected any. Specify a different type. (@typescript-eslint/no-explicit-any)
| @@ -124,7 +125,7 @@ export class BuilderCanvas extends Vue { | |||
|
|
|||
| get blockWidth(): number { | |||
| // it returns array as we loop blocks inside v-for | |||
| const blockElementRef = this.$refs[`block/${this.blockAtTheFurthestRightPosition?.uuid}`] as Vue[] | |||
| const blockElementRef = this.$refs[`block/${this.blockAtTheFurthestRightPosition?.uuid}`] as any[] | |||
|
|
|||
| if (!blockElementRef) { | |||
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.@typescript-eslint/strict-boolean-expressions
Severity: ERROR
File: src/components/interaction-designer/BuilderCanvas.vue L130
Unexpected object value in conditional. The condition is always true. (@typescript-eslint/strict-boolean-expressions)
| @@ -124,7 +125,7 @@ export class BuilderCanvas extends Vue { | |||
|
|
|||
| get blockWidth(): number { | |||
| // it returns array as we loop blocks inside v-for | |||
| const blockElementRef = this.$refs[`block/${this.blockAtTheFurthestRightPosition?.uuid}`] as Vue[] | |||
| const blockElementRef = this.$refs[`block/${this.blockAtTheFurthestRightPosition?.uuid}`] as any[] | |||
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.@typescript-eslint/no-explicit-any
Severity: WARN
File: src/components/interaction-designer/BuilderCanvas.vue L128
Unexpected any. Specify a different type. (@typescript-eslint/no-explicit-any)
| @@ -108,7 +109,7 @@ export class BuilderCanvas extends Vue { | |||
|
|
|||
| get blockHeight(): number { | |||
| // it returns array as we loop blocks inside v-for | |||
| const blockElementRef = this.$refs[`block/${this.blockAtTheLowestPosition?.uuid}`] as Vue[] | |||
| const blockElementRef = this.$refs[`block/${this.blockAtTheLowestPosition?.uuid}`] as any[] | |||
| if (!blockElementRef) { | |||
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.@typescript-eslint/strict-boolean-expressions
Severity: ERROR
File: src/components/interaction-designer/BuilderCanvas.vue L113
Unexpected object value in conditional. The condition is always true. (@typescript-eslint/strict-boolean-expressions)
| @@ -108,7 +109,7 @@ export class BuilderCanvas extends Vue { | |||
|
|
|||
| get blockHeight(): number { | |||
| // it returns array as we loop blocks inside v-for | |||
| const blockElementRef = this.$refs[`block/${this.blockAtTheLowestPosition?.uuid}`] as Vue[] | |||
| const blockElementRef = this.$refs[`block/${this.blockAtTheLowestPosition?.uuid}`] as any[] | |||
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.@typescript-eslint/no-explicit-any
Severity: WARN
File: src/components/interaction-designer/BuilderCanvas.vue L112
Unexpected any. Specify a different type. (@typescript-eslint/no-explicit-any)
| @@ -6,13 +6,13 @@ | |||
|
|
|||
| <script lang="ts"> | |||
| /* eslint-disable @typescript-eslint/explicit-module-boundary-types,@typescript-eslint/strict-boolean-expressions */ | |||
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.vue/max-len
Severity: ERROR
File: src/components/interaction-designer/Connection.vue L8
This line has a length of 116. Maximum allowed is 100. (vue/max-len)
| import Lang from '@/lib/filters/lang' | ||
| import {Prop} from 'vue-property-decorator' | ||
| import {mixins, Options} from 'vue-class-component' | ||
| import {Lang} from '@/lib/filters/lang' |
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.import/order
Severity: ERROR
File: src/components/interaction-designer/block-editors/AdvancedExitEditor.vue L36
`@/lib/filters/lang` import should occur after import of `vuex-class` (import/order)
| @@ -18,28 +18,28 @@ | |||
| </validation-message> | |||
|
|
|||
| <h6 class="mt-2"> | |||
| {{ 'flow-builder.advanced-exit-name' | trans }} | |||
| {{ trans('flow-builder.advanced-exit-name') }} | |||
| </h6> | |||
| <textarea | |||
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.vuejs-accessibility/form-control-has-label
Severity: ERROR
File: src/components/interaction-designer/block-editors/AdvancedExitEditor.vue L23
Each form element must have a programmatically associated label element. (vuejs-accessibility/form-control-has-label)
| getTestInputFrom(exitEditor?: Vue): HTMLInputElement { | ||
| return (exitEditor?.$refs.testExpressionInput as Vue).$refs.input as HTMLInputElement | ||
| getTestInputFrom(exitEditor?: any): HTMLInputElement { | ||
| return (exitEditor?.$refs.testExpressionInput as any).$refs.input as HTMLInputElement |
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.@typescript-eslint/no-unnecessary-type-assertion
Severity: ERROR
File: src/components/interaction-designer/block-editors/AdvancedExitsBuilder.vue L104
This assertion is unnecessary since it does not change the type of the expression. (@typescript-eslint/no-unnecessary-type-assertion)
| getTestInputFrom(exitEditor?: Vue): HTMLInputElement { | ||
| return (exitEditor?.$refs.testExpressionInput as Vue).$refs.input as HTMLInputElement | ||
| getTestInputFrom(exitEditor?: any): HTMLInputElement { | ||
| return (exitEditor?.$refs.testExpressionInput as any).$refs.input as HTMLInputElement |
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.@typescript-eslint/no-explicit-any
Severity: WARN
File: src/components/interaction-designer/block-editors/AdvancedExitsBuilder.vue L104
Unexpected any. Specify a different type. (@typescript-eslint/no-explicit-any)
| return exitEditor?.$refs.name as HTMLInputElement | ||
| } | ||
|
|
||
| getTestInputFrom(exitEditor?: Vue): HTMLInputElement { | ||
| return (exitEditor?.$refs.testExpressionInput as Vue).$refs.input as HTMLInputElement | ||
| getTestInputFrom(exitEditor?: any): HTMLInputElement { |
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.@typescript-eslint/no-explicit-any
Severity: WARN
File: src/components/interaction-designer/block-editors/AdvancedExitsBuilder.vue L103
Unexpected any. Specify a different type. (@typescript-eslint/no-explicit-any)
| @@ -97,12 +96,12 @@ export class AdvancedExitsBuilder extends mixins(Lang) { | |||
| input?.focus() | |||
| } | |||
|
|
|||
| getNameInputFrom(exitEditor?: Vue): HTMLInputElement { | |||
| getNameInputFrom(exitEditor?: any): HTMLInputElement { | |||
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.@typescript-eslint/no-explicit-any
Severity: WARN
File: src/components/interaction-designer/block-editors/AdvancedExitsBuilder.vue L99
Unexpected any. Specify a different type. (@typescript-eslint/no-explicit-any)
| @@ -86,7 +85,7 @@ export class AdvancedExitsBuilder extends mixins(Lang) { | |||
| } | |||
|
|
|||
| this.block_removeExit({blockId: this.block.uuid, exit}) | |||
| this.focusInputEl(this.getTestInputFrom(this.$refs.draftExitEditor as Vue)) | |||
| this.focusInputEl(this.getTestInputFrom(this.$refs.draftExitEditor as any)) | |||
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.@typescript-eslint/no-explicit-any
Severity: WARN
File: src/components/interaction-designer/block-editors/AdvancedExitsBuilder.vue L88
Unexpected any. Specify a different type. (@typescript-eslint/no-explicit-any)
| @@ -68,7 +67,7 @@ export class AdvancedExitsBuilder extends mixins(Lang) { | |||
|
|
|||
| this.$nextTick(() => { | |||
| const isNamePopulated = !isEmpty(this.draftExit?.name) | |||
| const lastExitEditor = last(this.$refs.exitEditors as Vue[]) | |||
| const lastExitEditor = last(this.$refs.exitEditors as any) | |||
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.@typescript-eslint/no-explicit-any
Severity: WARN
File: src/components/interaction-designer/block-editors/AdvancedExitsBuilder.vue L70
Unexpected any. Specify a different type. (@typescript-eslint/no-explicit-any)
| @@ -68,7 +67,7 @@ export class AdvancedExitsBuilder extends mixins(Lang) { | |||
|
|
|||
| this.$nextTick(() => { | |||
| const isNamePopulated = !isEmpty(this.draftExit?.name) | |||
| const lastExitEditor = last(this.$refs.exitEditors as Vue[]) | |||
| const lastExitEditor = last(this.$refs.exitEditors as any) | |||
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.@typescript-eslint/no-unsafe-argument
Severity: ERROR
File: src/components/interaction-designer/block-editors/AdvancedExitsBuilder.vue L70
Unsafe argument of type `any` assigned to a parameter of type `List<unknown> | null | undefined`. (@typescript-eslint/no-unsafe-argument)
| @@ -142,7 +142,7 @@ class Home extends mixins(Lang, Routes) { | |||
| @Getter isConfigured!: boolean | |||
|
|
|||
| async beforeCreate(): Promise<void> { | |||
| const {$store} = this | |||
| const {$store}: any = this | |||
|
|
|||
| forEach(store.modules, (v, k) => !$store.hasModule(k) && $store.registerModule(k, v)) | |||
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.@typescript-eslint/strict-boolean-expressions
Severity: ERROR
File: src/views/Home.vue L147
Unexpected any value in conditional. An explicit comparison or type cast is required. (@typescript-eslint/strict-boolean-expressions)
| @@ -142,7 +142,7 @@ class Home extends mixins(Lang, Routes) { | |||
| @Getter isConfigured!: boolean | |||
|
|
|||
| async beforeCreate(): Promise<void> { | |||
| const {$store} = this | |||
| const {$store}: any = this | |||
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.@typescript-eslint/no-explicit-any
Severity: WARN
File: src/views/Home.vue L145
Unexpected any. Specify a different type. (@typescript-eslint/no-explicit-any)
| import {forEach, isEmpty} from 'lodash' | ||
| import {mixins} from 'vue-class-component' | ||
| import {mixins, Options} from 'vue-class-component' | ||
| import {store} from '@/store' |
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.import/order
Severity: ERROR
File: src/views/Home.vue L117
`@/store` import should occur after import of `@floip/flow-runner` (import/order)
| @@ -109,18 +109,18 @@ | |||
| </template> | |||
|
|
|||
| <script lang="ts"> | |||
| import Lang from '@/lib/filters/lang' | |||
| import {Lang} from '@/lib/filters/lang' | |||
| import Routes from '@/lib/mixins/Routes' | |||
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.import/order
Severity: ERROR
File: src/views/Home.vue L113
`@/lib/mixins/Routes` import should occur after import of `@floip/flow-runner` (import/order)
| @@ -109,18 +109,18 @@ | |||
| </template> | |||
|
|
|||
| <script lang="ts"> | |||
| import Lang from '@/lib/filters/lang' | |||
| import {Lang} from '@/lib/filters/lang' | |||
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.import/order
Severity: ERROR
File: src/views/Home.vue L112
`@/lib/filters/lang` import should occur after import of `@floip/flow-runner` (import/order)
| </span> | ||
| <span | ||
| ref="extra-content-1" | ||
| :class="{'d-none': isExtraContentHidden}"> | ||
| {{ 'flow-builder.flow-builder-info-2' | trans }} | ||
| {{ trans('flow-builder.flow-builder-info-2') }} | ||
| </span> | ||
| </p> | ||
| <p | ||
| ref="extra-content-2" | ||
| :class="{'d-none': isExtraContentHidden}"> |
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.vue/object-curly-spacing
Severity: ERROR
File: src/views/Home.vue L77
A space is required after '{'. (vue/object-curly-spacing)
| </span> | ||
| <span | ||
| ref="extra-content-1" | ||
| :class="{'d-none': isExtraContentHidden}"> | ||
| {{ 'flow-builder.flow-builder-info-2' | trans }} | ||
| {{ trans('flow-builder.flow-builder-info-2') }} | ||
| </span> | ||
| </p> | ||
| <p | ||
| ref="extra-content-2" | ||
| :class="{'d-none': isExtraContentHidden}"> |
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.vue/object-curly-spacing
Severity: ERROR
File: src/views/Home.vue L77
A space is required before '}'. (vue/object-curly-spacing)
| </span> | ||
| <span> | ||
| <a | ||
| :class="{'d-none': !isExtraContentHidden}" | ||
| class="blue-link" | ||
| href="#" | ||
| @click="showExtraContent">{{ 'flow-builder.learn-more' | trans }}</a> | ||
| @click="showExtraContent">{{ trans('flow-builder.learn-more') }}</a> | ||
| </span> | ||
| <span | ||
| ref="extra-content-1" | ||
| :class="{'d-none': isExtraContentHidden}"> |
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.vue/object-curly-spacing
Severity: ERROR
File: src/views/Home.vue L71
A space is required after '{'. (vue/object-curly-spacing)
| </span> | ||
| <span> | ||
| <a | ||
| :class="{'d-none': !isExtraContentHidden}" | ||
| class="blue-link" | ||
| href="#" | ||
| @click="showExtraContent">{{ 'flow-builder.learn-more' | trans }}</a> | ||
| @click="showExtraContent">{{ trans('flow-builder.learn-more') }}</a> | ||
| </span> | ||
| <span | ||
| ref="extra-content-1" | ||
| :class="{'d-none': isExtraContentHidden}"> |
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.vue/object-curly-spacing
Severity: ERROR
File: src/views/Home.vue L71
A space is required before '}'. (vue/object-curly-spacing)
| @@ -57,44 +57,44 @@ | |||
| <div class="col-sm-7"> | |||
| <p> | |||
| <span> | |||
| {{ 'flow-builder.flow-builder-info-1' | trans }} | |||
| {{ trans('flow-builder.flow-builder-info-1') }} | |||
| </span> | |||
| <span> | |||
| <a | |||
| :class="{'d-none': !isExtraContentHidden}" | |||
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.vue/object-curly-spacing
Severity: ERROR
File: src/views/Home.vue L64
A space is required after '{'. (vue/object-curly-spacing)
| @@ -57,44 +57,44 @@ | |||
| <div class="col-sm-7"> | |||
| <p> | |||
| <span> | |||
| {{ 'flow-builder.flow-builder-info-1' | trans }} | |||
| {{ trans('flow-builder.flow-builder-info-1') }} | |||
| </span> | |||
| <span> | |||
| <a | |||
| :class="{'d-none': !isExtraContentHidden}" | |||
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.vue/object-curly-spacing
Severity: ERROR
File: src/views/Home.vue L64
A space is required before '}'. (vue/object-curly-spacing)
| </router-link> | ||
| <div class="mt-4"> | ||
| <router-link | ||
| :to="route('flows.importFlow')" | ||
| :title="trans('flow-builder.import-flow')" | ||
| class="mt-3 mr-2 active"> | ||
| {{ 'flow-builder.import-flow' | trans }} | ||
| {{ trans('flow-builder.import-flow') }} | ||
| </router-link> | ||
| </div> | ||
| <div class="mt-4"> | ||
| <language-adder v-slot="scope"> | ||
| <button |
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.vue/html-button-has-type
Severity: ERROR
File: src/views/Home.vue L31
Missing an explicit type attribute for button. (vue/html-button-has-type)
| @@ -128,7 +128,7 @@ class ImportFlow extends mixins(Lang, Routes) { | |||
| fileName = '' | |||
|
|
|||
| async beforeCreate(): Promise<void> { | |||
| const {$store} = this | |||
| const {$store}: any = this | |||
|
|
|||
| forEach(store.modules, (v, k) => !$store.hasModule(k) && $store.registerModule(k, v)) | |||
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.@typescript-eslint/strict-boolean-expressions
Severity: ERROR
File: src/views/ImportFlow.vue L133
Unexpected any value in conditional. An explicit comparison or type cast is required. (@typescript-eslint/strict-boolean-expressions)
| @@ -128,7 +128,7 @@ class ImportFlow extends mixins(Lang, Routes) { | |||
| fileName = '' | |||
|
|
|||
| async beforeCreate(): Promise<void> { | |||
| const {$store} = this | |||
| const {$store}: any = this | |||
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.@typescript-eslint/no-explicit-any
Severity: WARN
File: src/views/ImportFlow.vue L131
Unexpected any. Specify a different type. (@typescript-eslint/no-explicit-any)
| import {Getter, Mutation, namespace, State} from 'vuex-class' | ||
| import {debounce, forEach, get, isEmpty} from 'lodash' | ||
| import {mixins} from 'vue-class-component' | ||
| import {mixins, Options} from 'vue-class-component' | ||
| import {store} from '@/store' | ||
| import {IContext} from '@floip/flow-runner' | ||
| import ErrorHandlerV2 from '@/components/interaction-designer/flow-editors/import/ErrorHandlerV2.vue' |
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.import/order
Severity: ERROR
File: src/views/ImportFlow.vue L107
`@/components/interaction-designer/flow-editors/import/ErrorHandlerV2.vue` import should occur after import of `ajv` (import/order)
| import {Getter, Mutation, namespace, State} from 'vuex-class' | ||
| import {debounce, forEach, get, isEmpty} from 'lodash' | ||
| import {mixins} from 'vue-class-component' | ||
| import {mixins, Options} from 'vue-class-component' | ||
| import {store} from '@/store' |
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.import/order
Severity: ERROR
File: src/views/ImportFlow.vue L105
`@/store` import should occur after import of `ajv` (import/order)
| @@ -96,12 +96,12 @@ | |||
|
|
|||
| <script lang="ts"> | |||
|
|
|||
| import Lang from '@/lib/filters/lang' | |||
| import {Lang} from '@/lib/filters/lang' | |||
| import Routes from '@/lib/mixins/Routes' | |||
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.import/order
Severity: ERROR
File: src/views/ImportFlow.vue L100
`@/lib/mixins/Routes` import should occur after import of `ajv` (import/order)
| @@ -96,12 +96,12 @@ | |||
|
|
|||
| <script lang="ts"> | |||
|
|
|||
| import Lang from '@/lib/filters/lang' | |||
| import {Lang} from '@/lib/filters/lang' | |||
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.import/order
Severity: ERROR
File: src/views/ImportFlow.vue L99
`@/lib/filters/lang` import should occur after import of `ajv` (import/order)
| item.ussdContent, | ||
| identity, | ||
| ))) | ||
| const isEmptyClipboard = this.$store.state.trees.tree.hasClipboard && (isEmpty(item.clipboardContent) || isEmpty( | ||
| const isEmptyClipboard = this.$store.state["trees"].tree.hasClipboard && (isEmpty(item.clipboardContent) || isEmpty( |
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.dot-notation
Severity: ERROR
File: src/views/ResourceViewer.vue L372
["trees"] is better written in dot notation. (dot-notation)
| item.smsContent, | ||
| identity, | ||
| ))) | ||
| const isEmptyUssd = this.$store.state.trees.tree.hasUssd && (isEmpty(item.ussdContent) || isEmpty(pickBy( | ||
| const isEmptyUssd = this.$store.state["trees"].tree.hasUssd && (isEmpty(item.ussdContent) || isEmpty(pickBy( |
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.quotes
Severity: ERROR
File: src/views/ResourceViewer.vue L368
Strings must use singlequote. (quotes)
| item.smsContent, | ||
| identity, | ||
| ))) | ||
| const isEmptyUssd = this.$store.state.trees.tree.hasUssd && (isEmpty(item.ussdContent) || isEmpty(pickBy( | ||
| const isEmptyUssd = this.$store.state["trees"].tree.hasUssd && (isEmpty(item.ussdContent) || isEmpty(pickBy( |
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.dot-notation
Severity: ERROR
File: src/views/ResourceViewer.vue L368
["trees"] is better written in dot notation. (dot-notation)
| item.audioFiles, | ||
| identity, | ||
| ))) | ||
| const isEmptySMS = this.$store.state.trees.tree.hasSms && (isEmpty(item.smsContent) || isEmpty(pickBy( | ||
| const isEmptySMS = this.$store.state["trees"].tree.hasSms && (isEmpty(item.smsContent) || isEmpty(pickBy( |
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.quotes
Severity: ERROR
File: src/views/ResourceViewer.vue L364
Strings must use singlequote. (quotes)
| item.audioFiles, | ||
| identity, | ||
| ))) | ||
| const isEmptySMS = this.$store.state.trees.tree.hasSms && (isEmpty(item.smsContent) || isEmpty(pickBy( | ||
| const isEmptySMS = this.$store.state["trees"].tree.hasSms && (isEmpty(item.smsContent) || isEmpty(pickBy( |
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.dot-notation
Severity: ERROR
File: src/views/ResourceViewer.vue L364
["trees"] is better written in dot notation. (dot-notation)
| if (!this.hasContent(item.type)) { | ||
| return false | ||
| } | ||
| const isEmptyVoice = this.$store.state.trees.tree.hasVoice && (isEmpty(item.audioFiles) || isEmpty(pickBy( | ||
| const isEmptyVoice = this.$store.state["trees"].tree.hasVoice && (isEmpty(item.audioFiles) || isEmpty(pickBy( |
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.quotes
Severity: ERROR
File: src/views/ResourceViewer.vue L360
Strings must use singlequote. (quotes)
| if (!this.hasContent(item.type)) { | ||
| return false | ||
| } | ||
| const isEmptyVoice = this.$store.state.trees.tree.hasVoice && (isEmpty(item.audioFiles) || isEmpty(pickBy( | ||
| const isEmptyVoice = this.$store.state["trees"].tree.hasVoice && (isEmpty(item.audioFiles) || isEmpty(pickBy( |
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.dot-notation
Severity: ERROR
File: src/views/ResourceViewer.vue L360
["trees"] is better written in dot notation. (dot-notation)
| @@ -349,45 +353,45 @@ export class ResourceViewer extends mixins(Lang) { | |||
| // Empty content could be like: {} or {smsContent:{44:''} | |||
| // Generally, if a block is missing any of the content for all the content types | |||
| // that are enabled for the tree, the block should be considered “empty” and show up using this filter. | |||
| return filter(this.$store.state.trees.tree.blocks, (item) => { | |||
| return filter(this.$store.state["trees"].tree.blocks, (item) => { | |||
| if (!this.hasContent(item.type)) { | |||
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.@typescript-eslint/no-unsafe-argument
Severity: ERROR
File: src/views/ResourceViewer.vue L357
Unsafe argument of type `any` assigned to a parameter of type `string`. (@typescript-eslint/no-unsafe-argument)
| @@ -349,45 +353,45 @@ export class ResourceViewer extends mixins(Lang) { | |||
| // Empty content could be like: {} or {smsContent:{44:''} | |||
| // Generally, if a block is missing any of the content for all the content types | |||
| // that are enabled for the tree, the block should be considered “empty” and show up using this filter. | |||
| return filter(this.$store.state.trees.tree.blocks, (item) => { | |||
| return filter(this.$store.state["trees"].tree.blocks, (item) => { | |||
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.quotes
Severity: ERROR
File: src/views/ResourceViewer.vue L356
Strings must use singlequote. (quotes)
| @@ -349,45 +353,45 @@ export class ResourceViewer extends mixins(Lang) { | |||
| // Empty content could be like: {} or {smsContent:{44:''} | |||
| // Generally, if a block is missing any of the content for all the content types | |||
| // that are enabled for the tree, the block should be considered “empty” and show up using this filter. | |||
| return filter(this.$store.state.trees.tree.blocks, (item) => { | |||
| return filter(this.$store.state["trees"].tree.blocks, (item) => { | |||
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.dot-notation
Severity: ERROR
File: src/views/ResourceViewer.vue L356
["trees"] is better written in dot notation. (dot-notation)
| @@ -349,45 +353,45 @@ export class ResourceViewer extends mixins(Lang) { | |||
| // Empty content could be like: {} or {smsContent:{44:''} | |||
| // Generally, if a block is missing any of the content for all the content types | |||
| // that are enabled for the tree, the block should be considered “empty” and show up using this filter. | |||
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.vue/max-len
Severity: ERROR
File: src/views/ResourceViewer.vue L355
This line has a length of 109. Maximum allowed is 100. (vue/max-len)
| } | ||
|
|
||
| get tree(): any { | ||
| return this.$store.state.trees.tree | ||
| return this.$store.state["trees"].tree |
There was a problem hiding this comment.
Reporter: ESLint
Rule: eslint.rules.quotes
Severity: ERROR
File: src/views/ResourceViewer.vue L347
Strings must use singlequote. (quotes)
Context
Just a Experiment to identify the work needed to migrate flow-builder to vue 3
Summary
TODO list